is this specific to 145+ ? if so can you retain previous logic where applicable
e.g. #if TerrariaServer_1450_OrAbove .. then this new logic #else existing

future versions may switch compile time logic with whatever the input binary is, so it's useful to read this information directly in the patch.

Yep — Projectile.Damage_PVE_Inner is the 1.4.5 split from Projectile.Damage, so older versions still go through the existing Projectile.Damage branch.

This fix is just correcting the strike source to ldarg_0 (the projectile); the NPC parameter there is the hit target.

And yeah, wrapping that case in a 145+ #if makes the version split much clearer in the patch.

apologies for the confusion @CedaryCat - i was moreso stating if this patch is only relevant to 145+ then the conditional else would need to retain the old logic too. The case itself won't matter so much as it wont be present in older versions - however, I have actually just taken a look and now understand the actual issue.

I agree with the original patch where ldarg_0/this is passed through - at the moment its invoking an instance and passing through itself incorrectly.

No problem at all!

I've just pushed a new commit to restore the original patch. Let me know if everything looks good now.

No problem at all!

I've just pushed a new commit to restore the original patch. Let me know if everything looks good now.

Thank you!